Update custom model path instructions in DOCS.md#4369
Update custom model path instructions in DOCS.md#4369AJediIAm wants to merge 1 commit intohome-assistant:masterfrom
Conversation
Replace add-on with Home Assistant app. Home Assistant app is used instead of app for clarity.
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
sairon
left a comment
sairon
left a comment
There was a problem hiding this comment.
Please be clear about the scoping of the PR in the title.
| `/addon_configs/core_whisper/models/<your-model-dir>`. | ||
| Then, set the `custom_model` path to: | ||
| `/config/models/<your-model-dir>`. For a local model, the path must start with `/config/models/`, as this is how the add-on accesses your Home Assistant configuration directory through the container's mounted volume. | ||
| `/config/models/<your-model-dir>`. For a local model, the path must start with `/config/models/`, as this is how the Home Assistant app accesses your Home Assistant configuration directory through the container's mounted volume. |
There was a problem hiding this comment.
Just "app" is already used on line 13, and the title of the document says "Home Assistant App". Here it's just causing unnecessary cognitive overload.
| `/config/models/<your-model-dir>`. For a local model, the path must start with `/config/models/`, as this is how the Home Assistant app accesses your Home Assistant configuration directory through the container's mounted volume. | |
| `/config/models/<your-model-dir>`. For a local model, the path must start with `/config/models/`, as this is how the app accesses your Home Assistant configuration directory through the container's mounted volume. |
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
2 participants
Replace add-on with Home Assistant app. Home Assistant app is used instead of app for clarity.